Skip to content

Add Codex parity tooling#154

Merged
BASIC-BIT merged 3 commits into
mainfrom
codex/codex-parity-tooling
Jul 3, 2026
Merged

Add Codex parity tooling#154
BASIC-BIT merged 3 commits into
mainfrom
codex/codex-parity-tooling

Conversation

@BASIC-BIT

Copy link
Copy Markdown
Owner

Summary

  • add repo-local Codex MCP config and thin Codex skill wrappers for VRDex onboarding and VRCDN workflows
  • document Codex-specific workflow notes, tool mapping, and protected worktree policy
  • add protected-main guard hooks to root and workspace package scripts

Verification

  • node -e package JSON parse check for root, web, docs, and discord-gateway package files
  • node --check scripts/guard-main-worktree.mjs
  • D:\bench\VRDex\node_modules.bin\markdownlint-cli2
  • git diff --check HEAD~1..HEAD

@vercel

vercel Bot commented Jun 25, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
vr-dex-web Ready Ready Preview, Comment Jul 2, 2026 11:32pm

Request Review

@github-actions

github-actions Bot commented Jun 25, 2026

Copy link
Copy Markdown

Storybook Component Screenshot Preview

Outcome: success
Run: https://github.com/BASIC-BIT/VRDex/actions/runs/28628182516
Artifact: storybook-component-preview

Screenshots: primitive component stories captured on desktop and mobile.

This lane is separate from full-route Playwright screenshots and focuses on design-system component regressions.

@github-actions

github-actions Bot commented Jun 25, 2026

Copy link
Copy Markdown

Storybook Image Diff

Outcome: success
Run: https://github.com/BASIC-BIT/VRDex/actions/runs/28628182516
Artifact: storybook-image-diff

Changed Storybook baselines: none in this PR.

This check compares design-system component screenshots against committed baselines. Inline images show only added or modified Storybook baseline PNGs.

@greptile-apps

greptile-apps Bot commented Jun 25, 2026

Copy link
Copy Markdown

Greptile Summary

This PR adds Codex parity tooling and protected-main workflow guards. The main changes are:

  • Repo-local Codex MCP configuration for Playwright, Vercel, Convex, AWS, and Daytona tools.
  • Thin Codex skill wrappers for VRCDN and VRDex onboarding workflows.
  • Codex-specific agent documentation for MCP usage, skill mapping, and protected worktrees.
  • Protected-main guard lifecycle hooks across root, web, docs, and Discord gateway package scripts.
  • A Node guard script that blocks local workflow commands on main unless CI or an explicit bypass is active.

Confidence Score: 4/5

The workflow guard additions are mostly isolated, but one Codex skill wrapper references a missing source file and should be fixed before relying on the new VRCDN onboarding path.

The changed files are primarily configuration, package scripts, and documentation wrappers, with a narrow tooling failure confirmed around the VRCDN skill reference.

.codex/skills/vrcdn/SKILL.md

T-Rex T-Rex Logs

What T-Rex did

  • Validated a missing source skill condition by running a repo-local Node validation script that reads .codex/skills/vrcdn/SKILL.md and confirms it references .opencode/skills/vrcdn/SKILL.md; the validation failed because the referenced source skill does not exist on disk.
  • Inspected the protected-main guard workflow by comparing before and after artifacts, confirming that prehooks were added and that the main guard exits and non-main CI paths behaved as described.
  • Reviewed the Codex parity run by comparing before and after parity artifacts, identifying an MCP server name contract failure and listing missing MCP sections such as chrome-devtools, context7, and sequential-thinking.

View all artifacts

T-Rex Ran code and verified through T-Rex

Comments Outside Diff (1)

  1. General comment

    P1 Codex config omits documented MCP server names

    • Bug
      • The claimed Codex parity contract expects .codex/config.toml to expose the documented project MCP server names chrome-devtools, context7, playwright, and sequential-thinking. The executed head check parsed the actual [mcp_servers.*] sections and found only playwright, vercel_vrdex, convex, AWS-related entries, and daytona; chrome-devtools, context7, and sequential-thinking are missing.
    • Cause
      • The MCP server entries added to .codex/config.toml do not match the documented/validated server-name contract.
    • Fix
      • Update .codex/config.toml so its [mcp_servers.*] sections include the documented server names, at minimum [mcp_servers.chrome-devtools], [mcp_servers.context7], [mcp_servers.playwright], and [mcp_servers.sequential-thinking], or update the documentation/contract if these names are intentionally no longer valid.

    T-Rex Ran code and verified through T-Rex

Prompt To Fix All With AI
Fix the following 1 code review issue. Work through them one at a time, proposing concise fixes.

---

### Issue 1 of 1
.codex/skills/vrcdn/SKILL.md:9-10
**Missing source skill**
This wrapper routes Codex to `.opencode/skills/vrcdn/SKILL.md`, but the repository has no `.opencode/skills/**/SKILL.md` files. Opening the new Codex skill leads agents to a missing source of truth, so the wrapper cannot provide the advertised VRCDN guidance.

Reviews (1): Last reviewed commit: "Add Codex parity tooling" | Re-trigger Greptile

Comment thread .codex/skills/vrcdn/SKILL.md
@github-actions

github-actions Bot commented Jun 25, 2026

Copy link
Copy Markdown

Playwright Hosted Data-Flow

Outcome: success
Target: https://staging.vrdex.net
Hosted extended profile flow: enabled
Hosted auth helpers: enabled
Hosted adapter helpers: enabled
Run: https://github.com/BASIC-BIT/VRDex/actions/runs/28628182516
Artifact: playwright-hosted-data-flow

This optional check runs the mutation-backed profile flow against a configured hosted dev/staging target with isolated E2E test data.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: f6a1b37eac

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread scripts/guard-main-worktree.mjs Outdated
Comment thread .codex/config.toml Outdated
@github-actions

github-actions Bot commented Jun 25, 2026

Copy link
Copy Markdown

Playwright Public Screenshot Preview

Outcome: success
Run: https://github.com/BASIC-BIT/VRDex/actions/runs/28628182516
Artifact: playwright-public-preview

Screenshots: all public route checks passed on desktop and mobile.

Full screenshot set is available in the artifact. Pixel diff baselines are handled by the separate Playwright Image Diff check.

@github-actions

github-actions Bot commented Jun 25, 2026

Copy link
Copy Markdown

Playwright Image Diff

Outcome: success
Run: https://github.com/BASIC-BIT/VRDex/actions/runs/28628182516
Artifact: playwright-image-diff

Changed screenshot baselines: none in this PR.

This check compares public route screenshots against committed baselines. Inline images show only added or modified baseline PNGs.

@github-actions

github-actions Bot commented Jun 25, 2026

Copy link
Copy Markdown

Playwright Data-Flow Preview

Outcome: success
Run: https://github.com/BASIC-BIT/VRDex/actions/runs/28628182516
Artifact: playwright-data-flow

Captured flow:

  • test-gated profile submission form
  • gated helper rejection without the Playwright token
  • Convex profile creation
  • submission success state
  • public profile page readback
  • discovery search readback

Artifacts include screenshots, traces, and recorded video for the flow run.

@github-actions

github-actions Bot commented Jun 25, 2026

Copy link
Copy Markdown

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds repo-local Codex parity tooling and workflow guardrails for VRDex by introducing a Codex MCP config + thin skill wrappers, documenting Codex workflow notes, and enforcing a “protected main mirror” policy via npm/pnpm script guards.

Changes:

  • Add scripts/guard-main-worktree.mjs and wire it into root + workspace pre* scripts to block local execution on main (with explicit bypasses).
  • Add Codex documentation (docs/agentic/codex.md) and link it from the agentic docs index.
  • Add repo-local Codex scaffolding: .codex/config.toml, .codex/skills/* wrappers, and .codex/state/ gitignore.

Reviewed changes

Copilot reviewed 11 out of 12 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
scripts/guard-main-worktree.mjs New Node script that blocks running local workflows on main unless explicitly bypassed.
package.json Adds guard script and many pre* hooks so common root workflows invoke the guard.
docs/agentic/README.md Adds the Codex notes doc to the agentic docs index.
docs/agentic/codex.md Documents Codex workflow mapping, MCP config location, and protected-worktree policy.
apps/web/package.json Adds pre* guards for local web app scripts (dev/build/test/typecheck/etc.).
apps/docs/package.json Adds pre* guards for docs app scripts.
apps/discord-gateway/package.json Adds pre* guards for discord-gateway scripts.
AGENTS.local.md.example Extends the local agents example with a “worktree role” note.
.gitignore Ignores .codex/state/ alongside existing OpenCode state.
.codex/skills/vrdex-onboarding/SKILL.md New thin Codex wrapper pointing to the OpenCode onboarding skill.
.codex/skills/vrcdn/SKILL.md New thin Codex wrapper pointing to the OpenCode VRCDN skill with safety reminders.
.codex/config.toml New project-scoped Codex MCP server config mirroring the OpenCode server list.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread docs/agentic/codex.md
Comment thread .codex/config.toml Outdated
Comment thread scripts/guard-main-worktree.mjs
@BASIC-BIT BASIC-BIT merged commit 119cc47 into main Jul 3, 2026
22 checks passed
@BASIC-BIT BASIC-BIT deleted the codex/codex-parity-tooling branch July 3, 2026 00:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants